Skip to content

【PIR API adaptor No.133-135】Migrate paddle.logical_not/logical_or/logical_xor into pir - #58781

Merged
0x45f merged 4 commits into
PaddlePaddle:developfrom
ccsuzzh:logical_op
Nov 10, 2023
Merged

【PIR API adaptor No.133-135】Migrate paddle.logical_not/logical_or/logical_xor into pir#58781
0x45f merged 4 commits into
PaddlePaddle:developfrom
ccsuzzh:logical_op

Conversation

@ccsuzzh

@ccsuzzh ccsuzzh commented Nov 7, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor

PR types

Others

PR changes

APIs

Description

  • 新IR Python API适配升级 #58067
    No.133 No.134 No.135
    PIR API 推全升级
    将 paddle.logical_not 迁移升级至 pir，并更新单测，单测覆盖率：2/2
    将 paddle.logical_or 迁移升级至 pir，并更新单测，单测覆盖率：2/2
    将 paddle.logical_xor 迁移升级至 pir，并更新单测，单测覆盖率：2/2

  • test_type_error 单测暂不支持，数据类型检查的工作以后会下沉到 c++ 端去执行。

@paddle-bot paddle-bot Bot added the contributor External developers label Nov 7, 2023
@luotao1 luotao1 mentioned this pull request Nov 8, 2023
Closed
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Nov 8, 2023
@ccsuzzh

ccsuzzh commented Nov 8, 2023

Copy link
Copy Markdown
Contributor Author

image
@MarioLulab 我看到#58259中有类似问题，是将base改成static，可是这个ut里面本身就是static了，请问要怎么改呢？：）

@MarioLulab

MarioLulab commented Nov 8, 2023

Copy link
Copy Markdown
Collaborator

image @MarioLulab 我看到#58259中有类似问题，是将base改成static，可是这个ut里面本身就是static了，请问要怎么改呢？：）

这是个已知问题，目前的话建议最好统一采用 paddle.static.Program() 写法，而不是先前 import 的 Program。具体出现问题的原因内部还在排查。这个 pr 的单测里改成 paddle.static.Program() 写法吧，可以让 pr 先合入~

@ccsuzzh

ccsuzzh commented Nov 8, 2023

Copy link
Copy Markdown
Contributor Author

image @MarioLulab 我看到#58259中有类似问题，是将base改成static，可是这个ut里面本身就是static了，请问要怎么改呢？：）

这是个已知问题，目前的话建议最好统一采用 paddle.static.Program() 写法，而不是先前 import 的 Program。具体出现问题的原因内部还在排查。这个 pr 的单测里改成 paddle.static.Program() 写法吧，可以让 pr 先合入~

好的，感谢^v^

0x45f
0x45f reviewed Nov 9, 2023
View reviewed changes
Comment thread test/legacy_test/test_logical_op.py Resolved
@ccsuzzh
ccsuzzh requested a review from 0x45f November 10, 2023 02:14
@MarioLulab

MarioLulab commented Nov 10, 2023

Copy link
Copy Markdown
Collaborator

请问 test/legacy_test/test_logical_op.py 里的 test_type_error 单测没有切换成 pir 的模式的原因是？
如果 test_type_error 单测使用 test_with_pir_mode 会报错的话，辛苦在 pr 描述里说明一下哦，方便我们后续去排查 😄

@ccsuzzh

ccsuzzh commented Nov 10, 2023

Copy link
Copy Markdown
Contributor Author

请问 test/legacy_test/test_logical_op.py 里的 test_type_error 单测没有切换成 pir 的模式的原因是？ 如果 test_type_error 单测使用 test_with_pir_mode 会报错的话，辛苦在 pr 描述里说明一下哦，方便我们后续去排查 😄

会报错AssertionError: TypeError not raised by logical_and，具体如下图：
image

@MarioLulab

MarioLulab commented Nov 10, 2023

Copy link
Copy Markdown
Collaborator

请问 test/legacy_test/test_logical_op.py 里的 test_type_error 单测没有切换成 pir 的模式的原因是？ 如果 test_type_error 单测使用 test_with_pir_mode 会报错的话，辛苦在 pr 描述里说明一下哦，方便我们后续去排查 😄

会报错AssertionError: TypeError not raised by logical_and，具体如下图： image

数据类型检查的工作以后会下沉到 c++ 端去执行。这个 test_type_error 单测可以在 PR 描述里写一下暂不支持。然后跳过这个单测

@ccsuzzh

ccsuzzh commented Nov 10, 2023

Copy link
Copy Markdown
Contributor Author

请问 test/legacy_test/test_logical_op.py 里的 test_type_error 单测没有切换成 pir 的模式的原因是？ 如果 test_type_error 单测使用 test_with_pir_mode 会报错的话，辛苦在 pr 描述里说明一下哦，方便我们后续去排查 😄

会报错AssertionError: TypeError not raised by logical_and，具体如下图： image

数据类型检查的工作以后会下沉到 c++ 端去执行。这个 test_type_error 单测可以在 PR 描述里写一下暂不支持。然后跳过这个单测

Done

@0x45f
0x45f merged commit d4a612b into PaddlePaddle:develop Nov 10, 2023
@ccsuzzh
ccsuzzh deleted the logical_op branch November 13, 2023 06:57
danleifeng pushed a commit to danleifeng/Paddle that referenced this pull request Nov 14, 2023
@ccsuzzh
【PIR API adaptor No.133-135】Migrate paddle.logical_not/logical_or/log…
4c37217 
…ical_xor into pir (PaddlePaddle#58781)
SecretXV pushed a commit to SecretXV/Paddle that referenced this pull request Nov 28, 2023
@ccsuzzh @SecretXV
【PIR API adaptor No.133-135】Migrate paddle.logical_not/logical_or/log…
aaec791 
…ical_xor into pir (PaddlePaddle#58781)
@ccsuzzh ccsuzzh mentioned this pull request Dec 5, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
@0x45f 0x45f 0x45f approved these changes
@MarioLulab MarioLulab MarioLulab approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@luotao1 luotao1

@0x45f 0x45f

@YuanRisheng YuanRisheng

@MarioLulab MarioLulab

Labels

contributor External developers HappyOpenSource 快乐开源活动issue与PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@ccsuzzh @MarioLulab @0x45f @luotao1 @YuanRisheng